Skip to content

Comments

OMBC skeleton class implementation#68

Merged
VladIftime merged 18 commits intomainfrom
Dev-VladIftime-Kiflin-OMBC
Apr 13, 2025
Merged

OMBC skeleton class implementation#68
VladIftime merged 18 commits intomainfrom
Dev-VladIftime-Kiflin-OMBC

Conversation

@VladIftime
Copy link
Collaborator

No description provided.

@VladIftime VladIftime changed the title OMBC Skeleton Class OMBC skeleton class implementation Jan 8, 2025
@VladIftime VladIftime requested a review from jorritn January 8, 2025 11:24
@VladIftime VladIftime requested a review from wcoenraads January 24, 2025 11:47
Copy link
Collaborator

@wcoenraads wcoenraads left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This looks good to me! The only thing left to do is to add an OMBC definition to s2_control_type.py.

VladIftime and others added 2 commits March 3, 2025 10:21
@VladIftime VladIftime removed the request for review from jorritn March 3, 2025 09:25
Signed-off-by: Vlad Iftime <vladiftime60@gmail.com>
Signed-off-by: Vlad Iftime <vladiftime60@gmail.com>
@VladIftime VladIftime requested a review from wcoenraads March 3, 2025 15:17
Copy link
Collaborator

@wcoenraads wcoenraads left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This looks good - thanks!

@Flix6x
Copy link
Collaborator

Flix6x commented Apr 11, 2025

@VladIftime Could you resolve the conflicts and merge (or request a re-review) soon? I fear otherwise the merge conflicts might grow out of hand.

Signed-off-by: Vlad Iftime <vladiftime60@gmail.com>
Signed-off-by: Vlad Iftime <vladiftime60@gmail.com>
Signed-off-by: Vlad Iftime <vladiftime60@gmail.com>
Signed-off-by: Vlad Iftime <vladiftime60@gmail.com>
@VladIftime VladIftime merged commit 2d6119b into main Apr 13, 2025
19 checks passed
@VladIftime VladIftime deleted the Dev-VladIftime-Kiflin-OMBC branch April 13, 2025 12:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants